Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Advanced Juicer #1101

Closed

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented Mar 9, 2024

About the PR

Added advanced food service technology, with the new advanced reagent grinder

Why / Balance

The new advanced reagent grinder is the only grinder strong enough to crash artifact fragments with its new mighty blades.
It also comes with 6 slots vs 4 of the base machine, but with the cost of using 2 times more power.

This is required for balance related to obtaining liquid arti juice too fast and only using it to work on sci, making users work normally before they can spray to skip.

Technical details

Added a block in C# do stop processing any items that require an advanced grinder and show a warning.
This can be added to any item we want to require a strong grinder to be required for.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame,

image

8817c6ee-c507-49b3-995d-53d43c8f097a.mp4

Breaking changes

🆑 dvir01

  • add: Added advanced food service technology, with the new advanced reagent grinder.
  • tweak: The advanced reagent grinder is required for grinding artifact fragments.

Copy link
Contributor

github-actions bot commented Mar 9, 2024

RSI Diff Bot; head commit 730518f merging into cea9235
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_NF/Structures/Machines/advancedjuicer.rsi

State Old New Status
juicer0 Added
juicer1 Added

Edit: diff updated after 730518f

@Wolfking6116
Copy link
Contributor

It could be interesting to make some more advanced ores to make use of this. Some that have very light usage in the Ore Processor but could instead go through the grinder to provide a new source of some chemicals

@GreaseMonk
Copy link
Contributor

@dvir001 should increase grinder capacity for this one so it has a benefit of having it

@dvir001
Copy link
Contributor Author

dvir001 commented Mar 11, 2024

@dvir001 should increase grinder capacity for this one so it has a benefit of having it

You mean up the max size to 8?

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Mar 11, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Cheackraze
Copy link
Member

im definitely not opposed to the idea in theory, but I think that it needs more than just artifact juice to justify its existence. I think we could put some more of the silly grinder recipes in this category? like the plushie juices and batteries and stuff, and then maybe add new extractables to some more silly stuff?

@Wolfking6116
Copy link
Contributor

Oh I like that yeah. I've heard the plushie juices and stuff are being throw around lately with some reckless abandon so that could be a nice way to even it out.

Check, with chems being limited now what do you think of the idea of some new ores that give some chems, which could be ground up using this advanced grinder? Not at all common and certainly not giving large amounts of any chems. Thinking like equal to the same rarity as bananium with maybe 5-10 units of chemical per ore?

@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Mar 17, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Mar 18, 2024
@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Mar 25, 2024
@dvir001 dvir001 added Status: Awaiting Changes This PR has changes that need to be made before merging Status: DO NOT MERGE and removed Status: Needs Review This PR is awaiting reviews labels Apr 8, 2024
@dvir001
Copy link
Contributor Author

dvir001 commented Apr 8, 2024

Ill add more stuff to this.

Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Apr 19, 2024
@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Apr 27, 2024
@dvir001
Copy link
Contributor Author

dvir001 commented Jun 1, 2024

God help me on that one ill need to redo this

@dvir001 dvir001 closed this Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# FTL Sprites Status: Awaiting Changes This PR has changes that need to be made before merging Status: DO NOT MERGE YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants